Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plastex #95

Merged
merged 22 commits into from
Sep 24, 2014
Merged

Plastex #95

merged 22 commits into from
Sep 24, 2014

Conversation

paternal
Copy link
Contributor

Using plasTeX to render songs.

@paternal
Copy link
Contributor Author

Je vous laisse décider si cette branche doit être mergée ou non. Je l'ai créée pour centraliser les discussions à son propos.

@oliverpool
Copy link
Contributor

Assez partant pour merger,

Mais avant il faut:

  • decider de gérer ou non \cover
  • decider de gérer ou non \gtab (si oui, comment)
  • tester tout ça !

cf 0e98a78

@paternal
Copy link
Contributor Author

Et il y a des problèmes de chemins à résoudre : la liste des chansons s'affiche correctement chez moi, mais quand j'affiche les paroles d'une chanson, j'obtiens (avec SONGS_LIBRARY_DIR = "<patacrep>/patacrep/data/examples") :

Environment:


Request Method: GET
Request URL: http://127.0.0.1:8000/fr/songs/traditionnel/vent-frais/

Django Version: 1.6.1
Python Version: 2.7.8
Installed Applications:
('django.contrib.admin',
 'django.contrib.auth',
 'django.contrib.contenttypes',
 'django.contrib.sessions',
 'django.contrib.messages',
 'django.contrib.staticfiles',
 'django.contrib.sites',
 'south',
 'background_task',
 'captcha',
 'generator')
Installed Middleware:
('django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'django.middleware.locale.LocaleMiddleware')


Traceback:
File "<virtualenv>/local/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
  114.                     response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "<virtualenv>/local/lib/python2.7/site-packages/django/db/transaction.py" in inner
  339.                 return func(*args, **kwargs)
File "<virtualenv>/local/lib/python2.7/site-packages/django/views/generic/base.py" in view
  69.             return self.dispatch(request, *args, **kwargs)
File "<virtualenv>/local/lib/python2.7/site-packages/django/views/generic/base.py" in dispatch
  87.         return handler(request, *args, **kwargs)
File "<virtualenv>/local/lib/python2.7/site-packages/django/views/generic/detail.py" in get
  111.         context = self.get_context_data(object=self.object)
File "<patanet>/generator/views/songs.py" in get_context_data
  78.         context['content'] = _read_song(context['song'])
File "<patanet>/generator/views/songs.py" in _read_song
  63.     return parse_song(path)
File "<patanet>/generator/songs.py" in parse_song
  184.     song = parsetex(filename)
File "<patacrep>/patacrep/plastex.py" in parsetex
  94.     doc = SongParser.parse(filename)
File "<patacrep>/patacrep/plastex.py" in parse
  68.         tex.input(encoding.open_read(filename, 'r'))
File "<patacrep>/patacrep/encoding.py" in open_read
  20.             encoding=chardet.detect(open(filename, "r").read())['encoding'],

Exception Type: IOError at /fr/songs/traditionnel/vent-frais/
Exception Value: [Errno 2] No such file or directory: u'<patacrep>/patacrep/data/examples/songs/../patacrep/data/examples/songs/vent_frais.sg'

@oliverpool
Copy link
Contributor

Dans les settings, to chemin vers le dossier data devrait ressembler à un truc du genre:
SONGS_LIBRARY_DIR = os.path.join(settings.PROJECT_ROOT, "../patacrep/data/examples")
(ça m'étonnerai que tu aies un dossier '')

@paternal
Copy link
Contributor Author

Dans les settings, to chemin vers le dossier data devrait ressembler à un truc du genre:
SONGS_LIBRARY_DIR = os.path.join(settings.PROJECT_ROOT, "../patacrep/data/examples")
(ça m'étonnerai que tu aies un dossier '')

Dans l'erreur que j'ai donné, j'ai remplacé le chemin vers mes dossiers virtualenv, et les dépôts git de patacrep et patanet, par <virtualenv>, <patacrep> et <patanet>. J'ai la même définition de SONGS_LIBRARY_DIR que toi.

J'ai identifié l'erreur : #96.

@Luthaf
Copy link
Contributor

Luthaf commented Sep 1, 2014

Je suis pour merger aussi.

Le bug #96 sera géré séparément plus tard

decider de gérer ou non \cover
decider de gérer ou non \gtab (si oui, comment)

Je propose de se les garder pour plus tard, et de gérer les deux. Mais c'est plus un problème de front en fait ^^

tester tout ça !

Je vais m'y mettre =)

EDIT : ça m'a l'air bon au global. Quelques bug lorsque des intros sont indiqués avec des \musicnote comme ici, les deux points sont toujours là et semblent assez étranges.

@Luthaf
Copy link
Contributor

Luthaf commented Sep 23, 2014

Quelques bug lorsque des intros sont indiqués avec des \musicnote comme ici, les deux points sont toujours là et semblent assez étranges.

Je propose de merger et d'ouvrir des tickets pour la gestions des GTAB et des fichiers étranges. Le plus simple serait dans ce cas d'introduire les : dans la définition de \Intro.

Luthaf added a commit that referenced this pull request Sep 24, 2014
Plastex rendering for songs
@Luthaf Luthaf merged commit 7776b3b into master Sep 24, 2014
@Luthaf Luthaf deleted the plastex branch December 11, 2014 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants